-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(PC-33767) build(yarn): bump yarn #7471
Conversation
9fe89c8
to
e74ca30
Compare
Visit the preview URL for this PR (updated for commit 300fb0e): https://pc-native-testing--pr7471-bsr-bump-yarn-3-o8a979pe.web.app (expires Thu, 09 Jan 2025 11:44:36 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 54dd253190f9fbd3911a4f3fbe0f3af45c56e75f |
586c957
to
b23b948
Compare
b6b7fb9
to
bd00d00
Compare
bd00d00
to
300fb0e
Compare
Quality Gate passedIssues Measures |
Quality Gate passedIssues Measures |
@@ -21,7 +21,9 @@ eval "$(devbox generate direnv --print-envrc)" | |||
|
|||
layout node | |||
|
|||
source_env ./scripts/install_node_modules_when_not_installed.sh | |||
source ./scripts/load_certificate.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je pense que c'est inutile de le faire une seconde fois, c'est déjà faire ligne 12
@@ -21,7 +21,9 @@ eval "$(devbox generate direnv --print-envrc)" | |||
|
|||
layout node | |||
|
|||
source_env ./scripts/install_node_modules_when_not_installed.sh | |||
source ./scripts/load_certificate.sh | |||
export NODE_TLS_REJECT_UNAUTHORIZED=0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
j'imagine que si on montre ça coté sécu, il y aura un blocage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
source_env ./scripts/install_node_modules_when_not_installed.sh | ||
source ./scripts/load_certificate.sh | ||
export NODE_TLS_REJECT_UNAUTHORIZED=0 | ||
source ./scripts/install_node_modules_when_not_installed.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
source
est une fonction standard de bash
source_env
est une fonction standard de direnv
utilisant source
et permet de relancer le script s'il change
@@ -4,4 +4,4 @@ set -o errexit | |||
set -o nounset | |||
set -o pipefail | |||
|
|||
VERSION=$(yarn --silent json -f package.json version) | |||
VERSION=$(node -p "require('./package.json').version") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pour etre plus explicit et lisible, je recommande toujours les options en versions longues dans le code versionné
VERSION=$(node -p "require('./package.json').version") | |
VERSION=$(node --print "require('./package.json').version") |
"husky": { | ||
"hooks": { | ||
"pre-push": "yarn prepush" | ||
} | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pour quelles raisons il a été supprimé avec le bump de Yarn ?
Link to JIRA ticket: https://passculture.atlassian.net/browse/PC-33767
Flakiness
If I had to re-run tests in the CI due to flakiness, I add the incident on Notion
Checklist
I have:
Screenshots
delete if no UI change
Best Practices
Click to expand
These rules apply to files that you make changes to. If you can't respect one of these rules, be sure to explain why with a comment. If you consider correcting the issue is too time consuming/complex: create a ticket. Link the ticket in the code.as
(type assertions are removed at compile-time, there is no runtime checking associated with a type assertion. There won’t be an exception ornull
generated if the type assertion is wrong). In certain casesas const
is acceptable (for example when defining readonly arrays/objects). Usingas
in tests is tolerable.any
(when you want to accept anything because you will be blindly passing it through without interacting with it, you can useunknown
). Usingany
in tests is tolerable.!
when you know that the value can’t benull
orundefined
).@ts-expect-error
and@eslint-disable
.yarn test:lint
,yarn test:types
,yarn start:web
...).gap
(ViewGap
) instead of<Spacer.Column />
,<Spacer.Row />
or<Spacer.Flex />
.Test specific:
user
tofireEvent
.setFeatureFlags
. If not possible, mention which one(s) you want to mock in a comment (example:jest.spyOn(useFeatureFlagAPI, 'useFeatureFlag').mockReturnValue(true) // WIP_NEW_OFFER_TILE in renderPassPlaylist.tsx
)await act(async () => {})
andawait waitFor(/* ... */)
byawait screen.findBySomething()
.act
by default andwaitFor
as a last resort.Advice: